Skip to content

docs: fix basic-host setup instructions in quickstart#398

Merged
jonathanhefner merged 2 commits intomodelcontextprotocol:mainfrom
rafa-thayto:docs/fix-quickstart-basic-host-setup
Jan 29, 2026
Merged

docs: fix basic-host setup instructions in quickstart#398
jonathanhefner merged 2 commits intomodelcontextprotocol:mainfrom
rafa-thayto:docs/fix-quickstart-basic-host-setup

Conversation

@rafa-thayto
Copy link
Contributor

@rafa-thayto rafa-thayto commented Jan 29, 2026

Summary

  • Add missing npm install and npm run build steps at the ext-apps root before running the basic-host example
  • Without these steps, basic-host fails because it imports from ../../../dist/src/ which doesn't exist

Problem

Following the current quickstart instructions:

git clone https://github.com/modelcontextprotocol/ext-apps.git
cd ext-apps/examples/basic-host
npm install
npm start

Results in build failures because:

  1. The basic-host imports from ../../../dist/src/
  2. The dist directory doesn't exist until the root package is built
  3. Building requires dev dependencies (like ts-to-zod) that aren't installed when only running npm install in the example directory

Solution

Updated instructions to include root-level setup:

git clone https://github.com/modelcontextprotocol/ext-apps.git
cd ext-apps
npm install
npm run build
cd examples/basic-host
npm start

Test Plan

  • Follow the updated quickstart instructions from scratch
  • Verify basic-host starts successfully at http://localhost:8080
  • Verify the get-time tool works correctly

Add missing root-level npm install and build steps before running
the basic-host example. Without these steps, the example fails because
it imports from dist/src/ which doesn't exist until the root package
is built.

Fixes setup failure when following quickstart tutorial.
Copy link
Member

@jonathanhefner jonathanhefner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One nit. 🙏

Co-authored-by: Jonathan Hefner <jonathan@hefner.pro>
@rafa-thayto
Copy link
Contributor Author

One nit. 🙏

done! @jonathanhefner

@jonathanhefner jonathanhefner merged commit 90b5ac5 into modelcontextprotocol:main Jan 29, 2026
@jonathanhefner
Copy link
Member

Thank you, @rafa-thayto!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants